-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
x-pack/filebeat/input/httpjson/internal/v2: allow split chains to continue past empty targets #27880
Conversation
…tinue past empty targets This adds a configuration option "ignore_error" that allows a split processor chain to continue if a target field is present but empty. Updates elastic#26008
Pinging @elastic/security-external-integrations (Team:Security-External Integrations) |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
Trends 🧪💚 Flaky test reportTests succeeded. Expand to view the summary
Test stats 🧪
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Just dropped a thought
/test |
…tinue past empty targets (#27880) (#27914) This adds a configuration option "ignore_empty_value" that allows a split processor chain to continue if a target field is missing or empty. Updates #26008 (cherry picked from commit 2036ad8) Co-authored-by: Dan Kortschak <[email protected]> Co-authored-by: Dan Kortschak <[email protected]>
…tinue past empty targets (elastic#27880) This adds a configuration option "ignore_empty_value" that allows a split processor chain to continue if a target field is missing or empty. Updates elastic#26008
What does this PR do?
This adds a configuration option "ignore_error" that allows a split processor chain to continue if a target field is present but empty.
Why is it important?
This is required to address #26008 by allowing multiple splits on an input (see #26008 (comment)).
Checklist
- [ ] I have made corresponding change to the default configuration filesDefaults to existing behaviour.CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
split_test.go
How to test this PR locally
Standard testing.
Related issues
Use cases
See related issue.
Screenshots
N/A
Logs
N/A